Skip to content
This repository has been archived by the owner on Mar 22, 2018. It is now read-only.

added Dockerfile #65

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

added Dockerfile #65

wants to merge 3 commits into from

Conversation

charford
Copy link

@charford charford commented Dec 4, 2014

No description provided.

&& ln -sf /usr/local/share/phantomjs-1.9.7-linux-x86_64/bin/phantomjs /usr/bin/phantomjs

ADD . /app
RUN rm -rf /app/.git && cd /app && npm install
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why delete the .git?

@fzaninotto
Copy link
Owner

Could you add a line about this dockerfile in the README?

Casey Harford added 2 commits December 6, 2014 15:44
…irectory to screenshot-as-a-service, and removed maintainer line.
@charford
Copy link
Author

charford commented Dec 6, 2014

@fzaninotto thanks for reviewing my pull request. I went ahead and made the requested changes, as well as added some docker info to readme.

A note about the .git directory:

The reason I remove it during the build process(Dockerfile), is because that directory is inside the docker image. It's not necessary to keep it inside of the docker image because those files will never be modified directly. Instead, you would make changes to the files in your regular git repo outside of the docker image(your local file system), and then rebuild the docker image.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants